Conversation
newpavlov
left a comment
There was a problem hiding this comment.
Maybe also bump crate versions to make it clear that next releases will be breaking ones?
Unless I forgot something we didn't yet make any breaking changes to
So maybe I should revert the MSRV bump to core + RNG crates and add another CI MSRV test? I'll cross-post to the tracker for anyone watching that. |
|
|
|
AVR failure is rust-lang/rust#102278 ? |
|
Latest AVR failure:
At this point I'm tempted to remove AVR support. CC @Palladinium who added this CI test in #1144. |
|
Tests requiring unstable features are expected to fail from time to time, so maybe we should just ignore this failure? I expect it will be fixed eventually. |
|
Is there any point having a test if we ignore failures? Anyway, I was planning on leaving this open for a few more days. I added another PR to the list of pending breaking changes for |
|
Ideally, we would notice when it works again. However, in this case we pin the nightly, so I guess it won't fix itself. Does it make sense to keep the test as a comment? |
|
Okay, I commented the AVR test out. Lets go with a breaking release for |
|
Apparently AVR works on I won't bother re-enabling AVR (no motivation) but if someone else wants to, go ahead and make a PR. |
* Bump MSRV to 1.56 (Edition 2021) * Apply Clippy suggestions * Bump edition and add rust-version field to Cargo.toml * CI AVR test: unpin nightly rust version * Disable AVR test * Bump crate version numbers for a breaking release
We recently bumped the MSRV to 1.51 (#1246), but we could bump it further. This version is just over a year old, and the next release will be breaking anyway.
edition = "2021"andrust-version = "1.56"This PR also applies a few Clippy recommendations. Tempting to invoke rustfmt too, but there are too many open PRs which would conflict.
The README now notes:
If you object, please reply.